Skip to content

Conversation

@erlanggaramzy
Copy link
Owner

Checkpoint Rubric

This is the rubric that your instructor will use to grade your checkpoints. Please do not edit.

Checkpoint 1

  • All tests passed: 40 points
  • Proper use of documentation (commenting on code): 15 points
  • Properly indented code: 15 points
  • Demonstrated effective use of JavaScript: 30 points

Checkpoint 2

  • The application works as it should: 40 points
  • Proper use of documentation (commenting on code): 15 points
  • Properly indented code: 15 points
  • Demonstrated effective use of JavaScript and the DOM API: 30 points

Checkpoint 3

  • Use of React: 25 points
  • Accesses an API: 25 points
  • Proper use of documentation (commenting on code): 25 points
  • The application functions as it should: 25 points

Copy link

@reneemeyer reneemeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good start, see comments below. Also before merging please meet with a tutor or myself to correct pull request. It contains other files.

//take letters until first vowel and put to the end (pop) and add ay. addVowelAy()

function pigLatin(word) {
var wordArray = turnToList(word);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't use var, should be const

const checkValid = (word) => {
const arrayLength = word.length;
for (var i = 0; i < arrayLength; i++) {
if (isNaN(word[i])) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just return the evaluation


const checkConsonant = (word) => {
const vowel = ["a", "e", "i", "o", "u"];
var i = 0;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be let

const checkConsonant = (word) => {
const vowel = ["a", "e", "i", "o", "u"];
var i = 0;
var found = false;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be let

return false;
}

const checkConsonant = (word) => {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you need a check for constant and vowel?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants